Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhance: don't run travis-CI for only document change #2345

Merged
merged 1 commit into from
Nov 6, 2018
Merged

enhance: don't run travis-CI for only document change #2345

merged 1 commit into from
Nov 6, 2018

Conversation

fuweid
Copy link
Contributor

@fuweid fuweid commented Oct 23, 2018

save resource and time.

Signed-off-by: Wei Fu fuweid89@gmail.com

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗?)

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov
Copy link

codecov bot commented Oct 23, 2018

Codecov Report

Merging #2345 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2345      +/-   ##
==========================================
- Coverage    68.8%   68.72%   -0.09%     
==========================================
  Files         272      272              
  Lines       18175    18175              
==========================================
- Hits        12506    12491      -15     
- Misses       4253     4264      +11     
- Partials     1416     1420       +4
Flag Coverage Δ
#criv1alpha1test 31.54% <ø> (+0.02%) ⬆️
#criv1alpha2test 35.57% <ø> (-0.25%) ⬇️
#integrationtest 40.09% <ø> (-0.04%) ⬇️
#nodee2etest 33.04% <ø> (ø) ⬆️
#unittest 26.52% <ø> (ø) ⬆️
Impacted Files Coverage Δ
pkg/streams/utils.go 82.14% <0%> (-7.15%) ⬇️
apis/server/utils.go 71.15% <0%> (-3.85%) ⬇️
ctrd/watch.go 80.3% <0%> (-3.04%) ⬇️
ctrd/container.go 58.49% <0%> (-0.8%) ⬇️
cri/v1alpha2/cri.go 68.04% <0%> (-0.49%) ⬇️
daemon/mgr/container.go 58.79% <0%> (-0.22%) ⬇️
ctrd/image.go 78.94% <0%> (+2.19%) ⬆️

@fuweid fuweid changed the title enhance: don't run travis-CI for only document [wip] enhance: don't run travis-CI for only document Oct 23, 2018
@fuweid fuweid changed the title [wip] enhance: don't run travis-CI for only document [wip] enhance: don't run travis-CI for only document change Oct 23, 2018
INSTALLATION.md Outdated
@@ -189,7 +189,7 @@ This guide provides step by step instructions to deploy PouchContainer on bare m
As a developer, you may need to build and test PouchContainer binaries via source code. To build pouchd which is so-called "PouchContainer Daemon" and pouch which is so-called "PouchContainer CLI", the following build system dependencies are required:

* Linux Kernel 3.10+
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need to update this part, this part is never accurate.
CentOS, 3.10-327
Ubuntu, 4.0?

Could someone could help me to make it clear? @alibaba/pouch

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will do that. @allencloud

save resource and time.

Signed-off-by: Wei Fu <fuweid89@gmail.com>
@fuweid fuweid changed the title [wip] enhance: don't run travis-CI for only document change enhance: don't run travis-CI for only document change Nov 6, 2018
@zhuangqh
Copy link
Contributor

zhuangqh commented Nov 6, 2018

LGTM merged

@zhuangqh zhuangqh merged commit fbf37ed into AliyunContainerService:master Nov 6, 2018
@fuweid fuweid deleted the save_travis_ci_resource branch March 25, 2019 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants